Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[HOLD] move backoff settings, send email, and create client to config #1181

Closed
wants to merge 13 commits into from

Conversation

AlbertSnows
Copy link
Contributor

@AlbertSnows AlbertSnows commented Sep 3, 2024

PR on hold until #1182 is merged
Link to JIRA ticket if there is one: https://ctds-planx.atlassian.net/browse/PXP-11392

This PR moves config properties and functions out of utils into config. Utils should be low-level, and thus not reference other namespaces for sake of not inducing circular dependencies.

New Features

Breaking Changes

Bug Fixes

Improvements

  • move some config properties and variables out of utils into config

Dependency updates

Deployment changes

Copy link

github-actions bot commented Jan 7, 2025

Please find the ci env pod logs here

@AlbertSnows
Copy link
Contributor Author

closing for the rework here -> #1209

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant